Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add emoji categories to emoji list #232

Merged
merged 9 commits into from
Aug 30, 2023

Conversation

esensar
Copy link
Contributor

@esensar esensar commented Aug 28, 2023

This closes #229

This also closes a part of #155 - search is missing and will need some more work

@tibbi
Copy link
Contributor

tibbi commented Aug 29, 2023

it doesnt show up well on smaller screens, half of the first smiley is cut for me, cannot see it by scrolling either. And icon colors dont change well.

@esensar
Copy link
Contributor Author

esensar commented Aug 30, 2023

I have made it fit available size now, since scrolling does not work well in that part of the screen when using gesture navigation.

@tibbi
Copy link
Contributor

tibbi commented Aug 30, 2023

yeah, works better now, just can we maybe stop the scrolling when a category changes by clicking on it?

@tibbi
Copy link
Contributor

tibbi commented Aug 30, 2023

lets make the inactive categories use the same color as ABC and Clear in Material You theme, especially visible in light one. They should have slight color on them, not just white.

@tibbi
Copy link
Contributor

tibbi commented Aug 30, 2023

or you know what, lets actually use the text color in Material You theme too for inactive categories and also ABC/Clear, just like it is on the main keyboard

@tibbi
Copy link
Contributor

tibbi commented Aug 30, 2023

ok thanks, this looks better

@tibbi tibbi merged commit 25918d4 into SimpleMobileTools:main Aug 30, 2023
@esensar esensar deleted the feature/229-emoji-categories branch August 30, 2023 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Split emojis into categories
2 participants